Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix compiling gsad with libmicrohttp >= 0.9.71 #2625

Merged
merged 3 commits into from
Dec 14, 2020
Merged

Fix compiling gsad with libmicrohttp >= 0.9.71 #2625

merged 3 commits into from
Dec 14, 2020

Conversation

bjoernricks
Copy link
Contributor

@bjoernricks bjoernricks commented Dec 12, 2020

What:

Fix compiling gsad with libmicrohttp 0.9.71 and later

Why:

ibmicrohttp 0.9.71 changed the return value of the request handler callback functions. This causes a compiler warning with newer distributions like Ubuntu 20.10. Because we treat warnings as errors (-Wall) the compilation fails on such systems.

How:

Build gsad on Ubuntu 20.10 successfully.

Checklist:

libmicrohttp introduced a new enum for their http request handling
callback functions. Introducing this new return time causes several
compiler warnings and because we are treating warnings as errors the
whole gsad compilation did fail with libmicrohttp greater or equal than
0.9.71. This version of libmicrohttp is for example shipped with Ubuntu
20.10.

Fixes #2317
@bjoernricks bjoernricks requested a review from a team December 12, 2020 17:11
@codecov
Copy link

codecov bot commented Dec 12, 2020

Codecov Report

Merging #2625 (7f70b6a) into gsa-20.08 (073d2d2) will increase coverage by 0.30%.
The diff coverage is 74.26%.

Impacted file tree graph

@@              Coverage Diff              @@
##           gsa-20.08    #2625      +/-   ##
=============================================
+ Coverage      52.76%   53.06%   +0.30%     
=============================================
  Files           1070     1072       +2     
  Lines          25789    25900     +111     
  Branches        7312     7399      +87     
=============================================
+ Hits           13607    13744     +137     
+ Misses         11058    11034      -24     
+ Partials        1124     1122       -2     
Impacted Files Coverage Δ
gsa/src/gmp/commands/wizard.js 9.23% <0.00%> (ø)
gsa/src/gmp/models/tlscertificate.js 100.00% <ø> (ø)
gsa/src/web/components/error/errorpanel.js 100.00% <ø> (ø)
gsa/src/web/pages/alerts/condition.js 3.22% <0.00%> (ø)
gsa/src/web/pages/alerts/dialog.js 20.33% <0.00%> (ø)
.../web/pages/alerts/filtercountleastconditionpart.js 60.00% <ø> (ø)
gsa/src/web/pages/cpes/details.js 11.11% <ø> (ø)
gsa/src/web/pages/extras/trashactions.js 5.49% <0.00%> (ø)
gsa/src/web/pages/hosts/detailspage.js 23.80% <ø> (ø)
gsa/src/web/pages/hosts/dialog.js 50.00% <ø> (ø)
... and 69 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update df6db25...7f70b6a. Read the comment docs.

@swaterkamp swaterkamp merged commit 34cf277 into greenbone:gsa-20.08 Dec 14, 2020
@bjoernricks bjoernricks deleted the fix-compiling-gsad-with-libmicrohttp-0.9.71 branch December 14, 2020 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants